Revert "gtkwindow: Reset maximize / fullscreen_initially on state changes"

This reverts commit 862e0dab03e12c2fb1f6d99dd85c588dccc15dbc.

file chooser: Fix a crash

The save_widgets_create function was not safe against
being called multiple times in save mode, calling
gtk_file_chooser_set_create_folders was a way to trigger
this crash.

toolbutton: linkify properties

toolbutton: fix misleading docs

First only label-widget clashes with icon-name and not the label property. In
addtion fix the propertyname to use '-' instead of '_'.

gtk-demo: Add a filter model example

This shows computed columns.

fontchooserwidget: Refactor code so we actually optimize

Previously, we would pango_font_describe() every time the code ran and
we wouldn't ever hit the optimized quick exit.

The code now is a lot more complex because the
compute-actual-value-when-required-the-first-time approach is not
supported out of the box in GtkTreeModel (or GValue).

gtk3-demo: Don't set xalign of GtkCheckButton

It's deprecated and doesn't make a difference here.

GtkImage: Plug a memory leak

gtk-demo: Properly clean up timeouts in search entry demo

places sidebar: Don't leak shortcuts

Poined out by valgrind.

GtkSearchEngine: Be more robust in cancellation cases

When the search is cancelled, we may end up with a tracker
dbus reply coming in after the GtkSearchEngine object is
already gone, and bad things happen. Prevent this by
using g_signal_connect_object instead of g_signal_connect.

GtkQuery: Make API follow GTK+ conversions

We generally have const getters for strings, and the two users
of this API were promptly leaking the (unexpected) copy they
got from gtk_query_get_location and gtk_query_get_text.

places sidebar: Always chain up in finalize

The GtkSidebarRow class was forgetting to do this. The visible
symptom of this in valgrind is that you notice GObject data
being leaked.

places sidebar: Plug a memory leak

Updated Czech translation

Use versioned deprecations more seriously

I invadvertendly introduced a dependency on a recent GLib recently,
by cherry-picking a fix that used new GLib API. This commit will
help catching such errors before release, by using the versioned
deprecation machinery to turn such events into build-time
warnings.

places sidebar: Improve docs

Clarify the deprecation note for ::show-connect-to-server.
https://bugzilla.gnome.org/show_bug.cgi?id=752633

places sidebar: Improve the docs

Reword the docs of gtk_places_sidebar_set_drop_targets_visible().

places sidebar: Fix a copy-paste leftover

The ::populate-all nick and blurb where not referrering to this
property at all.
https://bugzilla.gnome.org/show_bug.cgi?id=752633

toolbutton: improve docs wrt stock deprecation

Docuemnt where to get the images from.

Updated Polish translation

Updated Hebrew translation

3.17.5

Fix distcheck

Makefile.decl does not work well with scripts that stay
in srcdir, so generate test-simplify from test-simplify.in,
just os that it ends up in builddir.

places view: Disconnect from volume monitor

Just unreffing the volume monitor object won't stop it from emitting
signals that we are connected to. Disconnect from the signals in
finalize.

reftests: Fix underlines in link-coloring reftest

Underlines now need to be manually specified in CSS since GTK got
text-decoration support.

treeview: Don't underallocate editing widgets that are too big

combobox: Unconditionally draw background and border

combobox: Remove autoscroll timer on popdown

combobox: Remove cell_view_frame

Cell views can draw their own frames now.

cellview: Implement border and background drawing

gtkimagemenuitem: improve the replacement docs even more

Since the stock strings usually have menmosics suggest
gtk_menu_item_new_with_mnemonic().

offscreen_window: Remove deprecated API calls

gtkimagemenuitem: fix suggestion for replacement

gtk_menu_item_new(9 would create an empty menu item. While the stock deprecation
aims to unclutter the menus, I feel this goes too far.

drawingarea: use same background approach as GtkLayout

Call gtk_style_context_set_background() every time the style is updated
and add a comment as to why we still need to do it here.

https://bugzilla.gnome.org/show_bug.cgi?id=752599

Revert "drawingarea: don't call gtk_style_context_set_background()"

This reverts commit 6786c9d724ad57ea30e169d2afcb4a314b2e56bb.

https://bugzilla.gnome.org/show_bug.cgi?id=752599

Revert "drawingarea: checks parent's draw() before call it"

This reverts commit b3166b745f41a901171821ccc9f0c69b9cc35e70.

https://bugzilla.gnome.org/show_bug.cgi?id=752599

file chooser: Memory handling fixes

I forgot to clean up the long press gesture, and in additon, creating
the rename popover from the ui template was causing the tree view
to not be disposed when the file chooser goes away. Work around this
by manually unsetting the relative-to widget of the popover in dispose.

Fix return value error in gtk_stack_get_interpolate_size

Updated Spanish translation

Updated Spanish translation

wayland: Always apply scaling factor to cursor hotspot and dimension

Prior to this patch, the hotspot would be passed in buffer coordinate
space. Where this were ever tested, i.e. in a patched mutter, the
server interpreted them incorrectly, which meant it went undiscovered.
In the updated mutter patches the incorrect behavior in GTK+ was
discovered due to the behavior in mutter was corrected.

In the themed cursor case, the dimensions were not correctly scaled
either, but this had no negative visible effect because the dimension is
only used for reporting damage tracking, and passing a bigger damage
region than surface has no negative visible effects.

https://bugzilla.gnome.org/show_bug.cgi?id=752616

GtkPopover: Set interpolate-size on the stack...

... in gtk_popover_bind_model
and gtk_popover_menu_init

GtkStack: Add interpolate-size property

And use it to determine whether we should actually interpolate between
stack sizes or not.

GtkListBox: Improve focus scrolling

We are trying to scroll the header in view together with the
focus row. The way this is implemented works fine when scrolling
up, but falls short when scrolling down. Fix this by making sure
that both the row and the header bar visible.

GtkListBox: Listen for adjustment changes

We automatically pick up an adjustment from our parent
scrollable, but we failed to update it when it changes.
This is happening in the places sidebar, and it was causing
the focus-tracking to fail there, letting the focus move
out of view. With this change, the focus remains visible.

places sidebar: Improve keynav

Remove the code in the sidebar keypress handler that imperfectly
reproduces what GtkListBox already does: moving selection and
focus in response to arrow keys.

trivial: Some reformatting

places sidebar: Remove pointless code

No need to set or unset adjustments explicitly.

spin button: Add Home and End as key bindings

Make these go to the lower/upper boundary of the adjustment, like
Ctrl-PageUp/PageDown already do.
https://bugzilla.gnome.org/show_bug.cgi?id=309300

GtkWidget: Add custom font map support

This allows to use a custom font map for parts of the widget
hierarchy - this can be used for application-specific fonts.

https://bugzilla.gnome.org/show_bug.cgi?id=751812

font chooser: Add placeholder text to preview entry

This will make things more understandable if an application
sets an empty preview text.
https://bugzilla.gnome.org/show_bug.cgi?id=725551

GdkGLContext little fix to property names and descriptions.

https://bugzilla.gnome.org/show_bug.cgi?id=752502

Updates for 3.17.5

builder-tool: Don't strip border-width

Strictly speaking, this would only have to be special-cased inside
GtkDialog, but lets just skip it altogether.

Fix csd decorations

By accident, commit b9b17c3c2a434983452561c3 removed one line
that wasn't unused, but important for hiding close buttons on
dialogs. Bring it back.

Updated Hebrew translation

window: Set the type hint when it changes, not just on map()

There is no reason to wait until map() to reset a type hint on the GDK
window. Just set it when it changes.

window: g_object_notify() on type-hint change

window: Don't update type hint when it didn't change

window: Allow setting type hint on mapped windows

window: Don't keep 2 type hints

1. This confuses the code as it's using the old type hint with the new
type hint on GDK window creation

2. It only existed as a workaround for old code that directly accessed
window->type_hint which hasn't been possible since 3.0.

Updated Portuguese translation

Updated Portuguese translation

Updated Czech translation

combobox: Simplify code

We don't add new children to the child's parent, we add them to ourself.
Because guess who a child's parent is?

combobox: Remove unused header function functionality

combobox: Remove unused argument from private function

Fix the build

Revert "GtkGrid: Add a few NULL checks"

This reverts commit e215db6da21fc7c54f49b247afabc06a18d60513.

We are actually requiring non-NULL out arguments here now.

testgtk: Remove pointless code

The display is getting overwritten immediately. Pointed out by
coverity.

inspector: Avoid a NULL deref

Coverity pointed out that getting a class name requires
dereferencing the pointer, so only don't do it if it is NULL.

GtkWindow: Fix a use-after-free

We were printing a string in a warning after freeing it.
Also, fix an x/y typo that might cause popovers to be
mispositioned.

file chooser: remove an unused GError variable

Pointed out by coverity

inspector: Remove some dead code

Not sure what I was thinking here. Coverity pointed this out.

inspector: Avoid a NULL deref

treeview: Add an explicit return for clarity

I believe that first_column can only ever be NULL here if
last_column is NULL too, in which case we'd exited already.
But coverity doesn't see that, so add an explicit exit.

tooltip: Remove unnecessary checks

Coverity uses these as clues to complain.

list box: Remove unnecessary checks

Again, coverity takes these as clues to complain.

GtkLabel: Remove an unnecessary check

Coverity has the annoying habit of guessing backwards that
if we check the value here, it might be NULL earlier.

GtkGrid: Add a few NULL checks

Coverity complained about these.

GtkActionHelper: Avoid null deref in debug code

gdk: Add some parameter checking

This may help coverity recognize that we're not called with
invalid combinations.

testdialog: Error out if templates are missing

Coverity complained about us ignoring the return value
here, and it is nicer to get an error message about this.

cups: Annotate a call whose return value we don't care about

We're closing this fd, but we're not going to do anything else
if that fails. Annotate to let coverity see this.

inspector: Check a return value

We don't care about the error here, but we were relying on
g_resources_get_info to zero the size even in case of error.
No need to do that, we can just check the return value. Plus,
it makes coverity happy.

inspector: Annotate a call whose return value we don't need

The way this code is written, we know that there is an item in
the hash table and the iter_next call will give it to us, so
no need to check the return value. Annotate the call to tell
coverity.

gtkicontheme: Check a return value

We were already looking at the error anyway, but rewriting things
this way lets coverity see the light.

gtkbuilderparser: Add some assertions

Add some assertions that things are non-NULL when we know they
are, so that coverity gets it.

gtk-demo: Check a return value

Found by coverity.

css: Fix GtkCssChange enum

Previously, the (1<<31) value evaluated to the 64bit value
  0xFFFFFFFF80000000
instead of
  0x0000000080000000
Avoid this by explicitly casting the value to unsigned long long.

Also cast all values to unsigned long long to achieve consistency
and hopefully in the future get new values added the same way.

headerbar: Switch css positions on direction change

Updated Hungarian translation

file chooser: Improve popover positioning

We keep the popover vertically tied to the selected row, but pick
up the horizontal position from the click / touch.

places sidebar: Preserve ::populate-popup functionality

We "fix" this signal the same way we did for GtkEntry and
GtkTextView: slightly change the signature so that the passed
widget may be a container other than a GtkMenu, and add a
property to explicitly opt in to receiving the signal in
this case.

file chooser: Use a popover for the context menu

Use a popover for the context menu, and add a long press gesture
to allow opening it with touch.

places sidebar: Use a popover for the context menu

Use a popover for the context menu, and add a long press gesture
to allow opening it with touch.

gdk: fix crash if visual is NULL

cairo_xlib_surface_get_visual might return NULL so we need to
check for NULL before trying to use it.

https://bugzilla.gnome.org/show_bug.cgi?id=752520

inspector: Avoid a gratitious use of g_strv_length

No need to count the strings first before iterating over them.

file chooser: Make bindings work in save entry

With the name entry being in the header bar now, we no longer
get bindings working by just letting the key event bubble up,
we have to explicitly apply them on key events that the save
entry is not handling.